Skip to content
This repository was archived by the owner on Mar 22, 2023. It is now read-only.

Backport appveyor update to stable-1.5#412

Merged
igchor merged 4 commits intopmem:stable-1.5from
igchor:stable_1.5_appveyor_update
Sep 9, 2019
Merged

Backport appveyor update to stable-1.5#412
igchor merged 4 commits intopmem:stable-1.5from
igchor:stable_1.5_appveyor_update

Conversation

@igchor
Copy link
Copy Markdown
Contributor

@igchor igchor commented Sep 6, 2019

This change is Reviewable

PMEM_IS_PMEM_FORCE is not set in tests
and Appveyor build is timed out.
... to reduce the Appveyor build time.
... to reduce the Appveyor build time.
@pmem-bot
Copy link
Copy Markdown
Contributor

pmem-bot commented Sep 6, 2019

@ldorau and @lukaszstolarczuk please review this pull request

Copy link
Copy Markdown
Member

@ldorau ldorau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 2 of 2 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

Copy link
Copy Markdown
Member

@ldorau ldorau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 6, 2019

Codecov Report

Merging #412 into stable-1.5 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           stable-1.5     #412   +/-   ##
===========================================
  Coverage       90.76%   90.76%           
===========================================
  Files              26       26           
  Lines            1050     1050           
===========================================
  Hits              953      953           
  Misses             97       97
Flag Coverage Δ
#tests_clang_debug_cpp17 91.44% <ø> (ø) ⬆️
#tests_clang_release 90.31% <ø> (ø) ⬆️
#tests_gcc_debug 93.39% <ø> (ø) ⬆️
#tests_gcc_release_cpp17 89.39% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7163200...929eb94. Read the comment docs.

@igchor igchor merged commit c7a04c2 into pmem:stable-1.5 Sep 9, 2019
@igchor igchor deleted the stable_1.5_appveyor_update branch October 7, 2019 13:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants